This is good to go, right?

not really, only @commitlint/format has non default export, rest of them have only default one

not really, only @commitlint/format has non default export, rest of them have only default one

Yeah. we should revert this . And format has two way to require:

const {format} = require('@commitlint/format')
// or
const format = require('@commitlint/format').default

@armano2 sorry! I only test format api, I didn't dig into other api module's source code

bdw, format is actually wierd, no clue why this is exported in this way

thats most likely reason why we have issues with this, package

@escapedcat Anyway, after revert. we are ready to go.

using .default syntax cloud not be wrong

bdw, format is actually wierd, no clue why this is exported in this way

commitlint/@commitlint/format/src/index.ts

Lines 1 to 2 in 31818ea

	export {default} from './format';
	export * from './format';

thats most likely reason why we have issues with this, package

Looks like test file counts on index.ts, that's why we need index.ts. If we change it to ./format(and change the corresponding package.json file), index.ts cloud be removed.

Am I right? @escapedcat @armano2

@AdeAttwood maybe we can just merged this?

@escapedcat yea I think merge this in, so the docs are correct.

I do like the destructuring syntax but could not get that to work. I think looking into that will be cool.

Thanks people!